Skip to content

Update 0.29.0 changelog regarding CommonJS imports following the file-loader update #757

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
May 9, 2020

Conversation

Lyrkan
Copy link
Collaborator

@Lyrkan Lyrkan commented May 9, 2020

As noted in #743 the file-loader update we did in 0.29.0 has a breaking change when files are imported using the CommonJS syntax.

Since the loader is now exporting ES modules by default, importing them with require(...) retrieves the whole module and not only the default export.

We'll also have to update the Github changelog after merging this (closes #743).

@Lyrkan Lyrkan requested a review from weaverryan May 9, 2020 10:07
@weaverryan
Copy link
Member

Thank you @Lyrkan! I'll update the release right now

@weaverryan weaverryan merged commit 2658151 into symfony:master May 9, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

dynamic image not found in react component
2 participants